checkbutton: Don't redundantly update states

States are set properly by GtkButton, there is no need to set them
manually.

checkbutton: Draw background unconditionally

We want to draw backgrounds always, not just on prelight. Themes can
decide to only set a background during prelight themselves.

reftests: Update for :active => :checked change

Updated Hebrew translation

stylecontext: Update docs to reflect theming engines being gone

themingengine: Deprecate

themingengine: Remove now unused private API

themingengine: Transition code towards StyleContext

We want to get rid of theming engines, so stop using them as much as
possible.

borderimage: Use style context as argument

... not the theming engine.

roundedbox: Remove functions that take a GtkThemingEngine

Instead, make the functions use a GtkStyleContext argument.

themingbackground: Remove initializer that takes a GtkThemingEngine

With the unification to use style contexts everywhere, make that
initializer the only one to use.

Also rename it appropriately.

stylecontext: Move private a11y function to private header

More updates

Update expected output for the font picker a11y test

This change is a consequence of the recent font chooser changes.

Update expected output in a11y tests for buttons

This is an expected change from the recent button state cleanups.

Fix make install

The path to gtk-update-icon-cache was set incorrectly in one case.

togglebutton: Remove unneeded code

The state is completely handled by gtkbutton.c so no need to do anything
in the togglebutton.

togglebutton: Don't duplicate button code

It's the same code, so no need to write the same vfunc twice.

radiobutton: :hover on hover

No special cases, those go in the theme.

button: Handle :inconsistent in GtkToggleButton

button: Handle :checked in gtktogglebutton.c

Don't try to handle it elsewhere.

widget: Add checks for setting state flags

This is more for GTK developers to catch when they forgot to change
GTK_STATE_FLAGS_BITS after adding a new state flag than to prevent
widget developers from using the wrong flags.

themingengine: Remove hack to draw backgrounds instead of checks

This was a hack we added in early 3.x to allow themes to customize their
checkmarks.
Now that we want to properly support real backgrounds everywhere,
supporting this feature would cause double draws of backgrounds.

stylecontext: Update docs for gtk_render_activity()

gtk: Don't use gtk_render_activity()

... in places where we draw a background. This was changed for GTK 3.0.0
to allow animations, but these days it doesn't make sense anymore to use
gtk_render_activity() for backgrounds.

button: Remove depressed_on_activate

All buttons should always be marked as :active when they are pressed.
That includes checkboxes (which are never activated in real code anyway,
so this change pretty much doesn't matter).

css: Do inconsistent checked states right

In HTML5, both pseudoclasses apply. So we do the same thing in our
widgets.

https://bugzilla.gnome.org/show_bug.cgi?id=733967

button: Remove depressed variable

It is not used anymore.

button: Don't honor child-displacement anymore

It doesn't make sense to support child displacement in a world where
pseudoclasses behave different from the actual displacement states.
Nobody would ever understand why a widget is displaced.

It is easily possible to simulate child displacement by using padding
CSS properties.

gtk: Use new :checked state

on:
- GtkToggleButton
- GtkCheckButton
- GtkRadioButton
- GtkModelButton
- GtkCellRendererToggle
- GtkCheckMenuItem

also update themes:
- Adwaita
- Raleigh
but not the win32 theme.

The new :checked state replaces :active for the actual checkedness of
the widgets and :active is now used exclusively while the button is being
pressed.

https://bugzilla.gnome.org/show_bug.cgi?id=733967

css: Add a :checked pseudoclass

https://bugzilla.gnome.org/show_bug.cgi?id=733967

Replace uses of g_memmove() by memmove()

g_memmove() is deprecated, it is a simple macro that just calls
memmove() with the same parameters.

Reviewed by Paolo Borelli on IRC.

Add an example for a hig-style toolbar

GtkFontChooser: Prevent font fallback in the preview

We don't want to preview the fallback fonts, after all.
https://bugzilla.gnome.org/show_bug.cgi?id=570414

Updated POTFILES.in

GtkFontButton: Avoid criticals

The font_family and font_face members may be NULL, and we
shouldn't cause critical warnings in that case.

GtkFontButton: Limit the decimals shown for size

It doesn't make sense to show a font size with 6 digits. Use
the same formatting we use in the font chooser widget now.

https://bugzilla.gnome.org/show_bug.cgi?id=724996

GtkFontChooserWidget: Allow fractional font sizes

With this change, we let users enter up to 2 decimals in the entry,
to specify font sizes exactly. The slider still rounds to whole
numbers.

https://bugzilla.gnome.org/show_bug.cgi?id=665711

GtkFontButton: Use the same font name as the font chooser

The font chooser constructs the display name for the font from
the family and face names. Do the same in the font button, so
we don't end up calling the same font by different names, which
would be confusing.

GtkFontChooserWidget: Use a better display name for the font

Instead of reconstructing a display name from the
PangoFontDescription, use the font family and face
objects, which have the original font. This lets us
get the names of fonts like Noto Sans CJK DemiLight
right, which would be shown as Noto Sans CJK SemiLight
when mangled via PangoFontDescription, since Pango
treats 'DemiLight' as an alias for the SemiLight weight.

https://bugzilla.gnome.org/show_bug.cgi?id=733832

GtkGestureSingle: Update declared defaults of properties

These should match the initial values, which were changed a
few commits ago. The defaultvalue test fails if there is a
mismatch, like here.

win32: Set the right source device

https://bugzilla.gnome.org/show_bug.cgi?id=734738

Deprecate GtkStatusIcon

GtkStatusIcon is using a problematic, XEmbed-based protocol under X,
and we want to get rid of it eventually. Document our intentions by
marking GtkStatusIcon as deprecated.

https://bugzilla.gnome.org/show_bug.cgi?id=734826

util/extract-strings: Use autotools syntax for building

Always refer to it as "extract-strings$(EXEEXT)" where needed to take
cross-compilation into account.

https://bugzilla.gnome.org/show_bug.cgi?id=731013

Revert "Bug 731013 - cross-compilation broken when building extract-strings"

This reverts commit 651d9e90e715ba0f4e246d03102cfb5353c19dc6.

The commit broke 'make dist' - the extract-strings sources are no
longer included in the tarball.

Updates for 3.13.7

GtkMenuItemAccessible: use "insert" signal

Use the "insert" rather than the "add" signal to reliably detect
additions of menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=372545

treeview: Make multipress gesture accept all buttons

That gesture is meant to handle clicks on multiple buttons, so unset
the GDK_BUTTON_PRIMARY default. Also, remove unnecessary boilerplate
with the new GtkGestureSingle/GtkEventController defaults.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

textview: Make multipress gesture accept all buttons

That gesture is meant to handle clicks on multiple buttons, so unset
the GDK_BUTTON_PRIMARY default. Also, remove unnecessary boilerplate
with the new GtkGestureSingle/GtkEventController defaults.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

range: Make multipress gesture accept all buttons

That gesture is meant to handle clicks from several buttons, so unset
the new GDK_BUTTON_PRIMARY default. Also, remove unnecessary boilerplate
with the new default values.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

window: Set GtkGestureSingle::button to 0 on multipress gesture

This gesture handles events from multiple buttons, so ensure the
gesture still does so after the default value change.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

scrolledwindow: Make kinetic scrolling only work on touch devices

This code was relying on the previous GtkGestureSingle::touch-only default
value, so restore the behavior back.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

spinbutton: Make the swipe gesture only work on touch devices

This code was relying on the previous GtkGestureSingle::touch-only default
value.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

label: Update to new GtkGestureSingle/GtkEventController defaults

GtkGestureSingle::button is set to 0 on the multipress gesture, as several
buttons are managed by that gesture. Also avoid some extra lines of code
setting what nowadays are default values.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

entry: Update to new GtkGestureSingle/GtkEventController defaults

GtkGestureSingle::button is set to 0 on the drag/multipress gestures, as several
buttons are managed by these gestures. Also, avoid some extra lines of code
setting what nowadays are default values.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

dnd: Set explicitly GtkGestureSingle::button to 0 on the drag gesture

This gesture handles drags done with any button, so unset the current button=1
default.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

colorswatch: Update to new GtkGestureSingle/GtkEventController defaults

GtkGestureSingle::button is set to 0 on the multipress gesture, as several
buttons are managed by that gesture. Also avoid some extra lines of code
setting what nowadays are default values.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

colorplane: Update to new GtkGestureSingle/GtkEventController defaults

GtkGestureSingle::button is set to 0, as multiple buttons are managed by
the same gesture. Also avoid some extra lines of code setting what nowadays
are default values.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

gesturesingle: Change default values of button and touch-only properties

This now defaults to sane values on mice, so touch-only is set to FALSE,
and button is set to GDK_BUTTON_PRIMARY;

https://bugzilla.gnome.org/show_bug.cgi?id=734285

window: Set multipress gesture phase to be GTK_PHASE_NONE explicitly

This used to rely on the default value, which has changed to a saner default.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

dnd: Set drag gesture propagation phase to GTK_PHASE_NONE explicitly

This used to rely on the default phase value.

https://bugzilla.gnome.org/show_bug.cgi?id=734285

eventcontroller: Default to GTK_PHASE_BUBBLE

This is in practice the most common value, so make that the default

https://bugzilla.gnome.org/show_bug.cgi?id=734285

Drop another unused bit of configury

pwd.h is not used anywhere in GTK+, so no need to check
for it.

Drop unused variable

Nothing is using gdk-pixbuf-csource anymore, so no need
to set up the GDK_PIXBUF_CSOURCE variable.

Fixed some object leaks in broadway_server_new() constructor.

https://bugzilla.gnome.org/show_bug.cgi?id=734778

Broadway: Make unix socket support conditional

We need to use gio/gunixsocketaddress.h and do so
only if HAVE_GIO_UNIX is defined.

BroadwayServer support for unix socket listening

At the present time broadway listens only for TCP/IP incoming
display connections. This patch implements the support for listening
on unix domain sockets too, adding the broadway_server_on_unix_socket_new()
constructor and the commandline option --unixsocket [path] to broadwayd.

https://bugzilla.gnome.org/show_bug.cgi?id=734420

Adapt to spinbutton changes

tests: Sanitize

gcc was unhappy again.

Updated POTFILES.skip

reftests: Add reftest for statusbar fix

https://bugzilla.gnome.org/show_bug.cgi?id=724281

Statusbar: Remove matching messages before popping

gtk_statusbar_remove_all() was popping the top message if its
context_id matched before removing other matching messages from the
stack. This meant that if the context_id of the second top message
matched it was still displayed when the top message was popped and
then removed from the list of messages without updating the display.
Fix this by removing all the matching messages below the top one
before popping it if it matches.

https://bugzilla.gnome.org/show_bug.cgi?id=724281

reftests: Make gcc happy

GtkListBoxRow: Set clip

https://bugzilla.gnome.org/show_bug.cgi?id=734709

Fix a thinko in creating event masks

&-ing together all the device event types yields zero,
which is clearly not the intention here. Use | instead,
to create the intended mask.

https://bugzilla.gnome.org/show_bug.cgi?id=734641

Add some spinbutton tests

These tests check some of the changes from the previous commits.

GtkSpinButton: Prevent public API from unsetting the adjustment

A spin button really needs an adjustment to work. So don't let the
user unset the adjustment by passing NULL to
gtk_spin_button_set_adjustment.

https://bugzilla.gnome.org/show_bug.cgi?id=734660

GtkSpinButton: Always emit value-changed when the adjustment is changed

Changing adjustment via the property setter would not emit
value-changed, however changing it via gtk_spin_button_configure would.

This inconsistency had the following side-effects:

  - Setting an adjustment with a different value would not update the
    value shown by the spin button.

  - Creating a spin button like this (common in GtkBuilder XML) will
    not show the initial value:
      g_object_new (GTK_TYPE_SPIN_BUTTON, "adjustment", adj, NULL);

Let's use the same code path (ie. gtk_spin_button_configure) for all
public facing API for setting adjustment. The code that handled the
details of swapping out the old adjustment with the new has been split
into an unset_adjustment method and the rest has been folded into
gtk_spin_button_configure.

A spin button really needs an adjustment to work, so we don't need
most of the NULL checks. However we do need to check in
unset_adjustment because setting a new adjustment during object
creation might try to unset a non-existent one.

https://bugzilla.gnome.org/show_bug.cgi?id=734660

Fix various warnings

https://bugzilla.gnome.org/show_bug.cgi?id=734737

Fix some print format warnings

https://bugzilla.gnome.org/show_bug.cgi?id=734736

Fix various warnings about unused things

https://bugzilla.gnome.org/show_bug.cgi?id=734735

gdkwayland-window: Don't map the grab transfer menu for GtkMenu

This is a bad idea in general, but it also means that the focus for a
window drops when we pop up a menu, since the grab transfer menu exists.

Updated Greek translation

Fix the build of the native gtk-update-icon-cache when cross-compiling.

As a noinst_PROGRAMS, the libtool generated for cross-compiling will be
used, which will mess up the linking. Create a all-local target instead.
Also ensure that building uses always a native version of the tool by
specifying a GTK_UPDATE_ICON_CACHE automake variable.
Finally "config.h" has been created to work for the target platform and
causes problem when cross-compiling. So we temporarily generate a basic
config.h which contains only the strict minimum.

Bug 731013 - cross-compilation broken when building extract-strings

It is actually a bad idea to use noinst_PROGRAMS for build tools,
because it adds a $(EXEEXT). It is best to override the all target
with all-local to trigger the tool build.

places-sidebar: Refine the spacing

Based on a patch by Yosef Or Boczko,
https://bugzilla.gnome.org/show_bug.cgi?id=731443

Adwaita: check/radio icon source override fix

Adwaita: app-notification revert erroneusly committed changes

Adwaita: Use -gtk-icon-source for checkmarks

instead of using background-image

css: Match state in siblings properly

Previously, we always returned the state flags of the original widget
and did not consider the siblings' pseudoclasses.

Testcase is attached.

widgetpath: Use CSS-like syntax in to_string()

Print pseudo-classes using ':' sign.

Adwaita: path-bar home button spacing

Adwaita: saner backdrop switch for the dark variant

Adwaita: saner menubar

GtkPathBar: Don't hardcode spacing in buttons

The theme will be changed to provide padding.

GtkPathBar: Make recoloring symbolics work

We were only recreating the icons the very first time the theme
changed, and not every time as was intended.

widget-factory: Avoid a compiler warning

This is adjusting to the draw_layer signature change.

widget-factory: Add a missing style class

Unfortunately, constructing image-only buttons by just adding
the image as a child does not give us the image-button style
class. Add it manually.

GtkTextView: Preserve the cairo context around draw_layer

Otherwise, draw_layer might unintentionally wreak havoc on the
regular drawing that happens between the two draw_layer calls.

GtkTextView: Change draw_layer signature

Vfuncs should really take 'this' of the right type - the type
that the vfunc is added in.